Skip to content

ENH: Use Release CMAKE_BUILD_TYPE for macOS Python CI builds#87

Closed
thewtex wants to merge 1 commit intoInsightSoftwareConsortium:python-buildsfrom
thewtex:macos-python-release-build
Closed

ENH: Use Release CMAKE_BUILD_TYPE for macOS Python CI builds#87
thewtex wants to merge 1 commit intoInsightSoftwareConsortium:python-buildsfrom
thewtex:macos-python-release-build

Conversation

@thewtex
Copy link
Member

@thewtex thewtex commented Nov 5, 2018

See the CONTRIBUTING guide. Specifically:

Start ITK commit messages with a standard prefix (and a space):

  • BUG: fix for runtime crash or incorrect result
  • COMP: compiler error or warning fix
  • DOC: documentation change
  • ENH: new functionality
  • PERF: performance improvement
  • STYLE: no logic impact (indentation, comments)
  • WIP: Work In Progress not ready for merge

Provide a short, meaningful message that describes the change you made.

When the PR is based on a single commit, the commit message is usually left as the PR message.

A reference to a related issue or pull request in your repository. You can automatically close a related issues using keywords

@mentions of the person or team responsible for reviewing proposed changes.

Thanks for contributing to ITK!

@hjmjohnson
Copy link
Member

@thewtex NOTE: Mac build is failing.

@thewtex thewtex requested a review from dzenanz November 5, 2018 21:13
Copy link
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was PythonGetNameOfClass the only test which was failing with MinSizeRel?

@thewtex
Copy link
Member Author

thewtex commented Nov 5, 2018

Changing from MinSizeRel to Release fixed two non-Python tests, itkGaussianInterpolateImageFunctionTest, itkLinearInterpolateImageFunctionTest.

E.g.

https://open.cdash.org/viewTest.php?onlyfailed&buildid=5614687

PythonGetNameOfClass is flaky on Linux and macOS -- disabling it #97 so we have the critical baseline of green builds across all three platforms to keep green.

@kwrobot kwrobot added the temp label Nov 6, 2018
@thewtex
Copy link
Member Author

thewtex commented Nov 6, 2018

Merging into master here:

#122

@thewtex thewtex closed this Nov 6, 2018
@thewtex thewtex deleted the macos-python-release-build branch November 6, 2018 14:52
@kwrobot kwrobot removed temp labels Nov 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants